🐛 Fix: users - club 연관관계 오류 수정 by imjuyongp · Pull Request #48 · Sportize/be

imjuyongp · 2026-02-04T07:35:42Z

#️⃣ Issue Number

✨Feat: 실시간 알림기능 #38

📝 요약(Summary)

문제점

leader 중복 관리: Club.leader 필드와 ClubMember.role=LEADER 두 곳에서 동호회장 관리
복합 유니크 제약조건 누락: 같은 사용자가 같은 동호회에 중복 가입 가능
User에 Club 관계 누락: 사용자가 가입한 동호회 조회 불편

DB 스키마 변경

-- clubs 테이블에서 leader_id 컬럼 제거됨
ALTER TABLE clubs DROP COLUMN leader_id;

-- club_members 테이블에 유니크 제약조건 추가됨
ALTER TABLE club_members ADD UNIQUE (club_id, user_id);

🛠️ PR 유형

어떤 변경 사항이 있나요?

버그 수정
코드 리팩토링

📸스크린샷 (선택)

💬 공유사항 to 리뷰어

erd 수정사항 있습니다.

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

커밋 메시지 컨벤션에 맞게 작성했습니다.
변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

Summary by CodeRabbit

릴리스 노트

개선 사항
- 클럽 리더 참조 방식이 변경되어 리더 확인과 멤버 관리가 더 명확해졌습니다.
- 사용자의 클럽 가입 내역 추적 기능이 추가되었습니다.
신규 기능
- 클럽 가입 요청을 생성·관리하는 요청 엔티티가 도입되었습니다.
- 다양한 유형의 알림(가입 요청, 승인/거절, 채팅, 댓글 등)과 읽음 처리 기능이 추가되었습니다.

coderabbitai · 2026-02-04T07:35:59Z

Caution

Review failed

The pull request is closed.

Walkthrough

Club 엔티티의 리더 관계를 User 직접 참조에서 ClubMember를 통한 간접 접근으로 리팩토링하고, Club 생성 API 시그니처 변경, User에 clubMemberships 추가, JoinClubRequest 및 Notification 엔티티 추가가 포함된 변경입니다.

Changes

Cohort / File(s)	Summary
Club 리더 관계 리팩토링 `src/main/java/com/be/sportizebe/domain/club/entity/Club.java`	`private User leader` 필드 제거. `getLeaderMember()`, `getLeader()`, `isLeader(Long userId)` 헬퍼 메서드 추가.
Club 생성 요청 API 변경 `src/main/java/com/be/sportizebe/domain/club/dto/request/ClubCreateRequest.java`	`toEntity(User, String)` → `toEntity(String)`로 시그니처 변경. User import 및 빌더에서 리더 설정 제거.
Club 서비스 적용 `src/main/java/com/be/sportizebe/domain/club/service/ClubServiceImpl.java`	변경된 `toEntity()` 호출 반영. 생성 흐름에서 리더를 ClubMember로 추가하는 로직 명시화. 리더 검증을 `club.isLeader(userId)`로 변경.
User 엔티티 확장 `src/main/java/com/be/sportizebe/domain/user/entity/User.java`	`@OneToMany(mappedBy = "user")` `clubMemberships` 필드 추가 (`List<ClubMember>`), 빌더 기본값 초기화.
Join 요청 엔티티 추가 `src/main/java/com/be/sportizebe/domain/notification/entity/JoinClubRequest.java`	`JoinClubRequest` 엔티티 추가(유니크 제약 user_id+club_id). 상태 enum(`PENDING`, `ACCEPTED`, `REJECTED`) 및 `accept()`, `reject()` 메서드 포함.
Notification 엔티티 추가 `src/main/java/com/be/sportizebe/domain/notification/entity/Notification.java`	`Notification` 엔티티 추가(수신자, 타입 enum, 메시지, isRead, joinClubRequest, targetId, markAsRead()). 테이블 인덱스 추가.

Sequence Diagram(s)

(생성된 변경은 엔터티 추가 및 리팩토링에 집중되어 명확한 다중 컴포넌트 순차 흐름이 정의되어 있지 않아 시퀀스 다이어그램을 생략합니다.)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

🐛 Fix: users - club 연관관계 오류 수정 #48 — 동일한 Club/User/ClubMember 리팩토링(Club.leader 제거, ClubCreateRequest 시그니처 변경, leader 로직 이동)과 직접적으로 연관되어 있음.

Poem

리더는 이제 토끼를 피해 걷지 않아요 🐰
멤버십 길로 슬며시 발자국 남기고,
관계는 가지를 따라 흐르네,
알림이 와서 문을 두드리면,
함께 모여 뛰놀자고 속삭여요. 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 44.44% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 사용자-클럽 관계 오류 수정이라는 주요 변경사항을 명확하게 설명합니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch feat/notification

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

imjuyongp added 2 commits February 4, 2026 16:20

🐛Fix: 동호회 leader관리 중복 저장 해결

5fd6abc

🐛Fix: users 엔티티에 club 연관관계 추가

24b42e8

coderabbitai bot approved these changes Feb 4, 2026

View reviewed changes

✨Feat: 알람기능 관련 엔티티 구현

e05a202

imjuyongp merged commit 2d95888 into develop Feb 4, 2026
1 check was pending

This was referenced Feb 4, 2026

✨Feat: 동호회 가입 신청, 실시간 알람 기능 구현 #50

Merged

♻️ Refactor: 실시간 알림기능 수정 #52

Merged

coderabbitai bot mentioned this pull request Feb 26, 2026

✨ Feat: 동호회 가입 신청서 작성 기능 추가 및 신청 조회 API 개선 #59

Merged

13 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🐛 Fix: users - club 연관관계 오류 수정#48

🐛 Fix: users - club 연관관계 오류 수정#48
imjuyongp merged 3 commits intodevelopfrom
feat/notification

imjuyongp commented Feb 4, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Feb 4, 2026 •

edited

Loading

Review failed

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

imjuyongp commented Feb 4, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

#️⃣ Issue Number

📝 요약(Summary)

문제점

🛠️ PR 유형

📸스크린샷 (선택)

💬 공유사항 to 리뷰어

✅ PR Checklist

Summary by CodeRabbit

릴리스 노트

Uh oh!

coderabbitai bot commented Feb 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review failed

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

imjuyongp commented Feb 4, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Feb 4, 2026 •

edited

Loading